Skip to content

fix(marketing): keep Grok mark clear of mobile hero copy - #4542

Open
NicL9923 wants to merge 1 commit into
pingdotgg:mainfrom
NicL9923:t3code/fix-issue-4234-screenshots
Open

fix(marketing): keep Grok mark clear of mobile hero copy#4542
NicL9923 wants to merge 1 commit into
pingdotgg:mainfrom
NicL9923:t3code/fix-issue-4234-screenshots

Conversation

@NicL9923

@NicL9923 NicL9923 commented Jul 25, 2026

Copy link
Copy Markdown

Summary

  • move the Grok mark into the open center position above the mobile hero headline
  • preserve the compact centered layout for viewports at or below 340px
  • leave desktop positioning unchanged

Fixes #4234

Before and after

Mobile (393 × 852)

Before After
Mobile hero before the fix, with the Grok mark overlapping the headline Mobile hero after the fix, with the Grok mark above the headline

Desktop (rendered at 1440 × 1000)

Before After
Desktop hero before the fix Desktop hero after the fix

The screenshots are hosted on the pr-assets-4234 branch in the fork so review evidence stays out of the product diff.

Verification

  • pnpm --filter @t3tools/marketing typecheck
  • pnpm --filter @t3tools/marketing build
  • git diff --check
  • integrated browser verification at 393 × 852 and 1440 × 1000
  • responsive overlap sweep at 320px, 340px, 393px, 430px, and 820px

Note

Low Risk
CSS-only marketing hero layout at two breakpoints; no logic, data, or desktop behavior changes.

Overview
Repositions the Grok floating harness icon on narrow viewports so it no longer sits over the mobile hero headline.

At ≤820px, .hf-grok moves from the left stack beside the title to the top row, centered above the headline (top: 44px, left: calc(50% - 39px)). The layout comment is updated to describe three marks above the headline and two by the CTA.

At ≤340px, Grok’s compact placement is adjusted (top: 57px, left: calc(50% - 26px)) so it stays centered with the smaller mark size. Desktop and other float marks are unchanged.

Reviewed by Cursor Bugbot for commit 858b935. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Reposition Grok floating mark to top-center on mobile hero

On mobile, the .hf-grok floating mark was overlapping the hero copy. It is now positioned near the top of the hero and centered horizontally using calc(50% - 39px), moving it away from the headline and CTA. For viewports ≤340px, a separate rule centers it at calc(50% - 26px) with a slightly lower top offset.

Macroscope summarized 858b935.

@coderabbitai

coderabbitai Bot commented Jul 25, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e04b3857-cdb3-4093-819d-c0e1af9f1029

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list. labels Jul 25, 2026
@macroscopeapp

macroscopeapp Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved

Pure CSS positioning changes adjusting where a decorative element appears on mobile views of the marketing page. No logic or runtime behavior changes - just visual layout adjustments.

You can customize Macroscope's approvability policy. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Grok logo overlays text on mobile website.

1 participant